Skip to content

Refactor: Renamed joint_reference_interfaces_ to reference_interface_… (backport #2008) #2012

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 1 commit into from
Nov 18, 2025
Merged

Refactor: Renamed joint_reference_interfaces_ to reference_interface_… (backport #2008) #2012

merged 1 commit into from
Nov 18, 2025

Conversation

@mergify
Copy link
Contributor

@mergify mergify bot commented Nov 18, 2025

Dear ros2_controllers maintainers, my previous pull request was taken down due to some issues. Because of the that issue I have to open a second pull request. I am deeply sorry for the inconvenience due to my error.

Furthermore, information related to this pull request are as following,

  1. Changed joint_reference_interfaces_ to reference_interface_name_ in 11 files with 31 additions and 31 deletions.

It passed following workflows,

  1. Check rolling compatibility on Jazzy
  2. Check rolling compatibility on Humble
  3. Check rolling compatibility on Kilted.

Again, I am extremely sorry for second pull request and your inconvenience.

This is an automatic backport of pull request #2008 done by [Mergify](https://mergify.com).

@codecov
Copy link

codecov bot commented Nov 18, 2025 •
edited
Loading

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 85.32%. Comparing base (fc5314c) to head (7ddb407).
⚠️ Report is 1 commits behind head on kilted.

Additional details and impacted files 
@@           Coverage Diff           @@
##           kilted    #2012   +/-   ##
=======================================
  Coverage   85.32%   85.32%           
=======================================
  Files         143      143           
  Lines       13936    13936           
  Branches     1201     1201           
=======================================
  Hits        11891    11891           
  Misses       1638     1638           
  Partials      407      407
Flag Coverage Δ
unittests 85.32% <100.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
...roller/test/test_ackermann_steering_controller.cpp 100.00% <100.00%> (ø)
...roller/test/test_ackermann_steering_controller.hpp 88.28% <ø> (ø)
...t/test_ackermann_steering_controller_preceding.cpp 100.00% <100.00%> (ø)
...ntroller/test/test_bicycle_steering_controller.cpp 100.00% <100.00%> (ø)
...ntroller/test/test_bicycle_steering_controller.hpp 85.71% <ø> (ø)
...est/test_bicycle_steering_controller_preceding.cpp 100.00% <100.00%> (ø)
...library/test/test_steering_controllers_library.cpp 100.00% <100.00%> (ø)
...library/test/test_steering_controllers_library.hpp 97.43% <ø> (ø)
...troller/test/test_tricycle_steering_controller.cpp 100.00% <100.00%> (ø)
...troller/test/test_tricycle_steering_controller.hpp 87.12% <ø> (ø)
... and 1 more
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@christophfroehlich christophfroehlich merged commit 787b2b7 into kilted Nov 18, 2025
14 checks passed
@christophfroehlich christophfroehlich deleted the mergify/bp/kilted/pr-2008 branch November 18, 2025 20:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@christophfroehlich @TheDevMystic